Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

fixes db deployment issue on heroku. production db value needs to be set... #1

Merged
merged 19 commits into from
Mar 27, 2014

Conversation

thehashrocket
Copy link
Contributor

fixes db deployment issue on heroku. production db value needs to be set in production.js and removed db variable from all.js.

@amoshaviv
Copy link
Contributor

Hi @jshultz this is a a good observation but we currently try to keep the config files clean from host specific properties. Definitely a problem though with the inclusion of Heroku's Mongo URIs and overriding them in the production file, will solve it soon.

@amoshaviv amoshaviv closed this Mar 27, 2014
@amoshaviv amoshaviv reopened this Mar 27, 2014
amoshaviv added a commit that referenced this pull request Mar 27, 2014
fixes db deployment issue on heroku. production db value needs to be set...
@amoshaviv amoshaviv merged commit acf1df5 into meanjs:master Mar 27, 2014
@amoshaviv
Copy link
Contributor

I merged this in and tweaked it a bit, thanks

@mpescarmona mpescarmona mentioned this pull request Mar 15, 2015
lirantal pushed a commit that referenced this pull request Jul 15, 2015
@xacaxulu xacaxulu mentioned this pull request Nov 20, 2015
pdfowler pushed a commit to pdfowler/mean that referenced this pull request Jan 19, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants